Skip to content

fix: Don't prevent rendering if buffer is empty#597

Closed
mrjones2014 wants to merge 1 commit intoMeanderingProgrammer:mainfrom
mrjones2014:mrj/push-orlyosyrnnwt
Closed

fix: Don't prevent rendering if buffer is empty#597
mrjones2014 wants to merge 1 commit intoMeanderingProgrammer:mainfrom
mrjones2014:mrj/push-orlyosyrnnwt

Conversation

@mrjones2014
Copy link

I think this: sudo-tee/opencode.nvim#200

Is actually a bug with this plugin.

If the text is non-empty, then is changed to empty via vim.api.nvim_buf_set_lines(buf, 0, -1, false, {}), render-markdown.nvim should still render in that case, otherwise it leaves behind some extmarks (I think those are extmarks)?

@MeanderingProgrammer
Copy link
Owner

This behavior is intentional and made to support LSP hover docs in older versions of neovim: 059f503

@mrjones2014
Copy link
Author

Hm okay interesting. Can you think of a different way to fix this issue?

Basically it leaves codeblock headers around if the buffer is cleared programmatically.

MeanderingProgrammer added a commit that referenced this pull request Feb 22, 2026
## Details

Issue: #597

Since we entirely skip rendering empty buffers if a buffer is cleared it
can cause extmarks to be left behind and not cleaned up.

Skipping empty buffers was added to fix how LSP hover buffers were
created, where they were initially empty, then later content was added.
However due to this order the debounce would skip rendering the latter.

To fix all of these interactions we now simply do not debounce when
rendering an empty buffer. As a result any decorations left should now
be properly cleaned up and content being added later should not get
skipped via the debounce.
@MeanderingProgrammer
Copy link
Owner

This should be fixed by: 1c95813

We now no longer skip rendering empty buffers and to still handle the old LSP hover docs case empty buffer renders are no longer debounced.

LMK if that resolves the issue!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants